Skip to content

Add Core Profile 11 Runner for Glassfish Embedded Web - #25446

Draft
Thihup wants to merge 19 commits into
eclipse-ee4j:mainfrom
Thihup:feature/tck-embedded-web
Draft

Add Core Profile 11 Runner for Glassfish Embedded Web#25446
Thihup wants to merge 19 commits into
eclipse-ee4j:mainfrom
Thihup:feature/tck-embedded-web

Conversation

@Thihup

@Thihup Thihup commented Apr 9, 2025

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread appserver/tests/tck/glassfish-runner/pom.xml Outdated
@OndroMih

Copy link
Copy Markdown
Contributor

Hi, @Thihup, this PR now passes the usual Jenkins CI. That doesn't run the Core TCK though.

What's the status? Does this runner run all the Core Profile TCKs? Do all tests pass?

@arjantijms

Copy link
Copy Markdown
Contributor

We have two Java files changes about a report, and the rest of the files is the usual copy of the ones in the platform-tck. We may think about whether we want to continuously copy these files, or just let them be at the platform-tck?

@dmatej
dmatej deleted the branch eclipse-ee4j:main December 17, 2025 15:32
@dmatej dmatej closed this Dec 17, 2025
@dmatej

This comment was marked as resolved.

@pzygielo pzygielo reopened this Jan 5, 2026
@pzygielo
pzygielo changed the base branch from 8.0 to main January 5, 2026 18:35
@arjantijms
arjantijms marked this pull request as ready for review January 5, 2026 19:43
@OndroMih
OndroMih marked this pull request as draft January 11, 2026 13:13
@OndroMih

Copy link
Copy Markdown
Contributor

It looks like this is just a work in progress and currently runs GlassFish Server and not Embedded. Turned it into a draft.

@arjantijms

Copy link
Copy Markdown
Contributor

It would be a great bennefit to have GlassFish Server run the core TCK as well.

Maybe we can split this in 2 issues? One for Server and one for Embedded?

Additionally there is the matter of copying all runners from https://github.com/jakartaee/platform-tck/tree/main/glassfish-runner or just leaving them there.

@arjantijms

Copy link
Copy Markdown
Contributor

See jakartaee/platform-tck#2715

A simple runner for GlassFish server

@renatsaf

renatsaf commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Hi @pzygielo ! Could you please check Jenkins?

@pzygielo

pzygielo commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Could you please check Jenkins?

I guess this can't start due to

image
21:16:42  ERROR: Maximum checkout retry attempts reached, aborting

Found unhandled java.io.IOException exception:

hudson.plugins.git.GitException: Command "git merge d47647eb6d95ac2193f10adf358e64cdf963bdfd" returned status code 1:

stdout: Auto-merging appserver/extras/embedded/web/src/assembly/package.xml

CONFLICT (content): Merge conflict in appserver/extras/embedded/web/src/assembly/package.xml

Auto-merging appserver/tests/tck/glassfish-runner/pom.xml

CONFLICT (add/add): Merge conflict in appserver/tests/tck/glassfish-runner/pom.xml

Auto-merging appserver/tests/tck/pom.xml

Auto-merging appserver/web/weld-integration/src/main/java/org/glassfish/cdi/persistence/PersistenceExtension.java

Auto-merging nucleus/admin/cli/src/main/java/com/sun/enterprise/admin/cli/embeddable/DeployerImpl.java

Automatic merge failed; fix conflicts and then commit the result.

OndroMih pushed a commit that referenced this pull request Jun 10, 2026
DeployerImpl.deploy(File, ...) returned normally even when the action
report contained failures, so callers could not detect a failed
deployment. Throw GlassFishException carrying the report message and
failure cause instead.

Extracted from #25446.
OndroMih pushed a commit that referenced this pull request Jun 10, 2026
afterBean() unconditionally dereferenced the current invocation, which
is absent when there is no default habitat or no active invocation (e.g.
GlassFish Embedded). Return early in those cases instead of throwing.

Extracted from #25446.
OndroMih pushed a commit that referenced this pull request Jun 10, 2026
A bundled META-INF/beans.xml in the embedded-web uber jar alters CDI
bean discovery of the assembled artifact. Exclude it from both unpacked
dependency sets.

Extracted from #25446.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants